Skip to content

[Service Bus] Add message batching#10305

Merged
ShivangiReja merged 4 commits intoAzure:masterfrom
ShivangiReja:Implement_TryAddMsg
Mar 5, 2020
Merged

[Service Bus] Add message batching#10305
ShivangiReja merged 4 commits intoAzure:masterfrom
ShivangiReja:Implement_TryAddMsg

Conversation

@ShivangiReja
Copy link
Copy Markdown
Member

The intent of these changes is to implement size-limited message batches and the means to send them to the Queue/Topic.

Goals

  • Add support for create batch feature
  • Add related tests
  • Cleanup code for send operation

Comment thread sdk/servicebus/Azure.Messaging.ServiceBus/src/Amqp/AmqpMessageConverter.cs Outdated
Comment thread sdk/servicebus/Azure.Messaging.ServiceBus/src/Core/TransportSender.cs Outdated
Comment thread sdk/servicebus/Azure.Messaging.ServiceBus/tests/ProcessorLiveTests.cs Outdated
Comment thread sdk/servicebus/Azure.Messaging.ServiceBus/tests/SenderLiveTests.cs Outdated
Comment thread sdk/servicebus/Azure.Messaging.ServiceBus/tests/SenderLiveTests.cs
Comment thread sdk/servicebus/Azure.Messaging.ServiceBus/src/Amqp/AmqpSender.cs Outdated
Comment thread sdk/servicebus/Azure.Messaging.ServiceBus/src/Amqp/AmqpSender.cs
Comment thread sdk/servicebus/Azure.Messaging.ServiceBus/tests/SenderLiveTests.cs
@JoshLove-msft
Copy link
Copy Markdown
Member

I wonder if we can copy the EH unit tests in AmqpEventBatchTests

@ShivangiReja
Copy link
Copy Markdown
Member Author

I wonder if we can copy the EH unit tests in AmqpEventBatchTests

Created an issue - #10325. Will add in a separate PR.

@ShivangiReja ShivangiReja merged commit 39aa434 into Azure:master Mar 5, 2020
@ShivangiReja ShivangiReja mentioned this pull request Mar 6, 2020
openapi-sdkautomation Bot pushed a commit to AzureSDKAutomation/azure-sdk-for-net that referenced this pull request Nov 27, 2020
Fix errors in various of readme files encountered in the recent go SDK release (Azure#11826)

* Fix an issue on windows

* Fix cdn

* Fix cognitiveservices/customVision

* Fix cognitiveservices/personalizer

* Fix cognitiveservices/translatorText

* Fix data plane digitaltwins

* Remove problematic tag from release - no impact on sdk, this has never been released

* amend Fix cognitiveservices/translatorText

* Fix imds

* Fix migrate readme that are mistakenly damaged in Azure#10305

* add one api-version back

* Fix digtaltwins data plane again

* Fix hybridnetwork

* Fix streamanlytics

* Revert some changes of digitaltwins

* Revert imds

* Fix migrate

* Fix migrate again
openapi-sdkautomation Bot pushed a commit to AzureSDKAutomation/azure-sdk-for-net that referenced this pull request Nov 27, 2020
Fix errors in various of readme files encountered in the recent go SDK release (Azure#11826)

* Fix an issue on windows

* Fix cdn

* Fix cognitiveservices/customVision

* Fix cognitiveservices/personalizer

* Fix cognitiveservices/translatorText

* Fix data plane digitaltwins

* Remove problematic tag from release - no impact on sdk, this has never been released

* amend Fix cognitiveservices/translatorText

* Fix imds

* Fix migrate readme that are mistakenly damaged in Azure#10305

* add one api-version back

* Fix digtaltwins data plane again

* Fix hybridnetwork

* Fix streamanlytics

* Revert some changes of digitaltwins

* Revert imds

* Fix migrate

* Fix migrate again
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants